-
Notifications
You must be signed in to change notification settings - Fork 561
Remove Plausible analytics integration #7525
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
WalkthroughThis change removes all tracking, analytics, and external script integrations related to Plausible and a third-party script from the dashboard, playground-web, and portal applications. It eliminates related dependencies, plugins, providers, and script tags from configuration files and layout components, without altering any public APIs or exported entities. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant DashboardApp
participant PlausibleProxy
participant ThirdwebScript
Note over User,DashboardApp: Previous Flow (Dashboard)
User->>DashboardApp: Load page
DashboardApp->>PlausibleProxy: Initialize Plausible tracking
DashboardApp->>User: Render page with PlausibleProvider
Note over User,DashboardApp: New Flow (Dashboard)
User->>DashboardApp: Load page
DashboardApp->>User: Render page (no Plausible tracking)
Note over User,Playground/Portal: Previous Flow (Playground/Portal)
User->>Playground/Portal App: Load page
Playground/Portal App->>ThirdwebScript: Load external script
Playground/Portal App->>User: Render page with script tag
Note over User,Playground/Portal: New Flow (Playground/Portal)
User->>Playground/Portal App: Load page
Playground/Portal App->>User: Render page (no external script)
📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
💤 Files with no reviewable changes (4)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms (7)
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
size-limit report 📦
|
09debda
to
d3ab94c
Compare
ce3aec8
to
cf261a6
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7525 +/- ##
=======================================
Coverage 51.97% 51.97%
=======================================
Files 952 952
Lines 64212 64212
Branches 4239 4238 -1
=======================================
Hits 33375 33375
Misses 30730 30730
Partials 107 107
🚀 New features to boost your workflow:
|
d3ab94c
to
fa3c37a
Compare
Merge activity
|
# Remove Plausible Analytics Integration This PR removes the Plausible Analytics integration from our applications: - Removed `next-plausible` dependency from the dashboard app - Removed Plausible script tags from dashboard, playground-web, and portal layouts - Removed the Plausible proxy configuration from the Next.js config - Updated the webpack configuration to work without the Plausible proxy wrapper <!-- start pr-codex --> --- ## PR-Codex overview This PR removes the usage of `next-plausible` from various files and updates dependencies accordingly. It also cleans up the code by removing unnecessary `<head>` elements and script imports across multiple layout files. ### Detailed summary - Removed `next-plausible` dependency from `apps/dashboard/package.json`. - Deleted `<Script>` components related to Plausible from `layout.tsx` files in `apps/playground-web`, `apps/dashboard`, and `apps/portal`. - Cleaned up `next.config.ts` by removing `withPlausibleProxy`. - Updated `pnpm-lock.yaml` to reflect the removal of `next-plausible`. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Removed Plausible analytics integration and related dependencies from the dashboard. * Removed external script loading from the layouts of the playground-web and portal apps. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
fa3c37a
to
9fba016
Compare
Remove Plausible Analytics Integration
This PR removes the Plausible Analytics integration from our applications:
next-plausible
dependency from the dashboard appPR-Codex overview
This PR focuses on removing the
next-plausible
package from various files in the codebase, including configuration and layout files, and updates dependency references inpackage.json
andpnpm-lock.yaml
.Detailed summary
next-plausible
dependency fromapps/dashboard/package.json
,pnpm-lock.yaml
.PlausibleProvider
component fromapps/dashboard/src/app/(app)/layout.tsx
andapps/portal/src/app/layout.tsx
.Script
imports and their usage in multiple layout files.next.config.ts
to excludewithPlausibleProxy
configuration.Summary by CodeRabbit